Skip to content

fix(ci): use shared refactor workflow automation - #2005

Merged
aidandaly24 merged 3 commits into
mainfrom
fix/refactor-shared-workflow-callers
Aug 25, 2026
Merged

fix(ci): use shared refactor workflow automation#2005
aidandaly24 merged 3 commits into
mainfrom
fix/refactor-shared-workflow-callers

Conversation

@aidandaly24

@aidandaly24 aidandaly24 commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Description

Restore refactor PR automation through shared DevTools workflows.

  • include refactor in CodeQL and PR automation triggers
  • consolidate Harness, Security, and PR Size/Title into one local pr-automation.yml
  • replace the repository-owned Harness implementation with a shared reusable call
  • pin Harness and Security jobs to aws/agentcore-devx-devtools#2
  • keep CLI-specific prompts and Harness image configuration local
  • remove the baked GitHub API credential from future Harness images
  • publish Harness reviews through the existing AgentCore DevX GitHub App

This deletes the local Harness invocation/publishing script; shared mechanics
now live in agentcore-devx-devtools.

Related Issue

N/A - workflow repair for the unreleased refactor branch.

Documentation PR

N/A

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

How have you tested the change?

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

Additional verification:

  • npm run format:check
  • parsed every workflow YAML file
  • git diff --check

Shared workflow verification is documented in aws/agentcore-devx-devtools#2.

Checklist

  • I have read the CONTRIBUTING document
  • Shared action tests cover the publishing fix
  • I have updated the documentation accordingly
  • No new example is needed for this CI-only change
  • My changes generate no new warnings
  • Dependency aws/agentcore-devx-devtools#2 must merge

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the
terms of your choice.

@aidandaly24
aidandaly24 requested a review from a team August 14, 2026 20:14
@github-actions github-actions Bot added size/m PR size: M agentcore-harness-reviewing AgentCore Harness review in progress labels Aug 14, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 14, 2026
@github-actions

github-actions Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.28.0.tgz

How to install

gh release download pr-2005-tarball --repo aws/agentcore-cli --pattern "*.tgz" --dir /tmp/pr-tarball
npm install -g /tmp/pr-tarball/aws-agentcore-0.28.0.tgz

@agentcore-devx-automation

Copy link
Copy Markdown
Contributor

Claude Security Review: no high-confidence findings. (run)

@agentcore-devx-automation agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label Aug 14, 2026
@github-actions github-actions Bot removed the agentcore-harness-reviewing AgentCore Harness review in progress label Aug 14, 2026
@github-actions

github-actions Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 40.99% 15453 / 37694
🔵 Statements 40.24% 16473 / 40935
🔵 Functions 35.15% 2644 / 7522
🔵 Branches 34.18% 10311 / 30161
Generated in workflow #4401 for commit 252dd8f by the Vitest Coverage Report Action

@github-actions github-actions Bot added size/l PR size: L and removed size/m PR size: M size/l PR size: L labels Aug 14, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 14, 2026
@agentcore-devx-automation

Copy link
Copy Markdown
Contributor

Claude Security Review: no high-confidence findings. (run)

@agentcore-devx-automation agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label Aug 14, 2026
# Conflicts:
#	.github/harness/README.md
#	.github/workflows/pr-ai-review.yml
#	examples/AgentCoreCliReviewer/app/PRReviewer/harness_review.py
@github-actions github-actions Bot added size/m PR size: M and removed size/l PR size: L labels Aug 24, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 24, 2026
@agentcore-devx-automation

Copy link
Copy Markdown
Contributor

Claude Security Review: no high-confidence findings. (run)

@agentcore-devx-automation agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label Aug 24, 2026
@aidandaly24
aidandaly24 merged commit cad9470 into main Aug 25, 2026
33 checks passed
@aidandaly24
aidandaly24 deleted the fix/refactor-shared-workflow-callers branch August 25, 2026 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/m PR size: M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants